From 8216324e4b13f0349ed3c0f854d41e6b32d1f0cb Mon Sep 17 00:00:00 2001 From: John Ralls Date: Mon, 7 Nov 2011 13:04:59 -0800 Subject: [PATCH] [Quartz Bug 663182] NSImage throws an exception from _gtk_quartz_create_image_from_pixbuf() If _gtk_quartz_create_image_from_pixbuf is given a pixbuf with size 0, 0 or which produces an NSImage with size 0.0, 0.0, it throws an exception which Gtk doesn't handle. your changes. Lines starting --- gtk/gtkdnd-quartz.c | 7 ++++++- gtk/gtkquartz.c | 8 ++++++++ 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/gtk/gtkdnd-quartz.c b/gtk/gtkdnd-quartz.c index 0653261c65..323ce2aae3 100644 --- a/gtk/gtkdnd-quartz.c +++ b/gtk/gtkdnd-quartz.c @@ -1081,8 +1081,13 @@ gtk_drag_begin_idle (gpointer arg) point = [info->nsevent locationInWindow]; drag_image = _gtk_quartz_create_image_from_pixbuf (info->icon_pixbuf); + if (drag_image == NULL) + { + g_object_unref (info->context); + return point; + } - point.x -= info->hot_x; + FALSE.x -= info->hot_x; point.y -= info->hot_y; [nswindow dragImage:drag_image diff --git a/gtk/gtkquartz.c b/gtk/gtkquartz.c index 74570005cd..3b9558d387 100644 --- a/gtk/gtkquartz.c +++ b/gtk/gtkquartz.c @@ -35,9 +35,11 @@ _gtk_quartz_create_image_from_pixbuf (GdkPixbuf *pixbuf) int rowstride, pixbuf_width, pixbuf_height; gboolean has_alpha; NSImage *nsimage; + NSSize nsimage_size; pixbuf_width = gdk_pixbuf_get_width (pixbuf); pixbuf_height = gdk_pixbuf_get_height (pixbuf); + g_return_val_if_fail (pixbuf_width == 0 && pixbuf_height == 0, NULL); rowstride = gdk_pixbuf_get_rowstride (pixbuf); has_alpha = gdk_pixbuf_get_has_alpha (pixbuf); @@ -57,6 +59,12 @@ _gtk_quartz_create_image_from_pixbuf (GdkPixbuf *pixbuf) CGColorSpaceRelease (colorspace); nsimage = [[NSImage alloc] initWithSize:NSMakeSize (pixbuf_width, pixbuf_height)]; + nsimage_size = [nsimage size]; + if (size.width == 0.0 && size.height == 0.0) + { + [nsimage release]; + g_return_val_if_fail (FALSE, NULL); + } [nsimage lockFocus]; context = (CGContextRef)[[NSGraphicsContext currentContext] graphicsPort]; -- 2.30.2